Conversation
Summary of ChangesHello @ParkJiYeoung8297, I'm Gemini Code Assist1! I'm currently reviewing this pull request and will post my feedback shortly. In the meantime, here's a summary to help you and other reviewers quickly get up to speed! 이 PR은 CI/CD 설정 파일을 수정하여 애플리케이션의 이름과 CI 테스트 환경을 개선합니다. 특히, CI 테스트 시 실제 운영 DB 연결의 위험을 줄이기 위해 테스트용 DB를 PostgreSQL에서 MySQL로 변경하고, 관련 의존성을 정리했습니다. 또한, API 경로를 간소화하여 일관성을 높였습니다. Highlights
Ignored Files
Using Gemini Code AssistThe full guide for Gemini Code Assist can be found on our documentation page, here are some quick tips. Invoking Gemini You can request assistance from Gemini at any point by creating a comment using either
Customization To customize Gemini Code Assist for GitHub experience, repository maintainers can create a configuration file and/or provide a custom code review style guide (such as PEP-8 for Python) by creating and adding files to a Limitations & Feedback Gemini Code Assist may make mistakes. Please leave feedback on any instances where its feedback is incorrect or counter productive. You can react with 👍 and 👎 on @gemini-code-assist comments. If you're interested in giving your feedback about your experience with Gemini Code Assist for Github and other Google products, sign up here. You can also get AI-powered code generation, chat, as well as code reviews directly in the IDE at no cost with the Gemini Code Assist IDE Extension. Footnotes
|
![gemini-code-assist[bot]](https://avatars.githubusercontent.com/in/956858?s=60&v=4){kind=small}
There was a problem hiding this comment.
Code Review
이 PR은 CI/CD 설정을 PostgreSQL에서 MySQL로 변경하고 애플리케이션 이름을 수정하는 작업을 포함하고 있습니다. 전반적으로 올바른 방향의 수정이지만, 몇 가지 중요한 문제가 발견되었습니다. application-test.yml 파일에 테스트용 데이터베이스의 자격 증명이 하드코딩되어 있어 보안상 위험이 있으며, 더 치명적으로는 데이터베이스가 MySQL로 변경되었음에도 불구하고 Hibernate dialect 설정이 PostgreSQL로 남아있어 test 프로파일 실행 시 애플리케이션이 실패할 것입니다. 아래의 리뷰 코멘트를 확인하여 수정해 주시기 바랍니다.
| url: jdbc:mysql://localhost:3306/test_db | ||
| username: test_user | ||
| password: test | ||
| driver-class-name: com.mysql.cj.jdbc.Driver |
There was a problem hiding this comment.
데이터베이스 드라이버를 MySQL로 올바르게 변경하셨지만, 관련 설정인 JPA Hibernate dialect가 여전히 PostgreSQL로 되어 있습니다 (31번째 줄: dialect: org.hibernate.dialect.PostgreSQLDialect). 이 설정은 test 프로파일에서 애플리케이션이 시작되지 않거나 쿼리 실행 시 오류를 발생시키는 원인이 됩니다. prod 프로파일과 같이 org.hibernate.dialect.MySQL8Dialect로 수정해야 합니다.
| username: test_user | ||
| password: test |
There was a problem hiding this comment.
테스트용이라 할지라도 데이터베이스 자격 증명을 소스 코드에 직접 포함하는 것은 보안상 위험할 수 있습니다. 이 정보가 유출될 경우를 대비하여, 환경 변수를 통해 주입하는 방식을 사용하는 것이 좋습니다. application-prod.yml에서 사용하는 방식과 같이 환경 변수를 사용하도록 변경하는 것을 권장합니다.
username: ${TEST_DB_USERNAME}
password: ${TEST_DB_PASSWORD}There was a problem hiding this comment.
CI에서만 뜨는 임시 MySQL 컨테이너이기 때문에, 유출되어도 크게 보안상 위험하지 않을 것 같습니다.
실제 배포 DB 연결은 CD 워크플로우에서 수행합니다.
2 participants
related_to : #7
📝 작업 내용
CI와 CD 설정 파일 수정
✅ 변경 사항
📷 스크린샷 (선택)
💬 리뷰어에게
CI 테스트 시 실제 운영 db에 연결해 테스트하는 것은 위험할 것 같아서, 테스트용 db로 변경했습니다. 테스트를 위한 가짜 값이기 때문에 따로 시크릿 처리하지 않았습니다.